Look at both GDK_SCROLL_MASK and GDK_BUTTON_PRESS_MASK. (#139628)

2006-05-09  Tor Lillqvist  <tml@novell.com>

	* gdk/win32/gdkevents-win32.c (doesnt_want_scroll): Look at both
	GDK_SCROLL_MASK and GDK_BUTTON_PRESS_MASK. (#139628)

diff --git a/ChangeLog b/ChangeLog
index 252d22cc108cbaf690c4822525502af5fd0c51de..a0cf0ce8f6e996505fee76eaed68aa551c3b1414 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-05-09  Tor Lillqvist  <tml@novell.com>
+
+       * gdk/win32/gdkevents-win32.c (doesnt_want_scroll): Look at both
+       GDK_SCROLL_MASK and GDK_BUTTON_PRESS_MASK. (#139628)
+
 2006-05-09  Matthias Clasen  <mclasen@redhat.com>
 
        * gtk/gtktextbuffer.c (gtk_text_buffer_class_init): Add a :cursor-position

diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index 252d22cc108cbaf690c4822525502af5fd0c51de..a0cf0ce8f6e996505fee76eaed68aa551c3b1414 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,8 @@
+2006-05-09  Tor Lillqvist  <tml@novell.com>
+
+       * gdk/win32/gdkevents-win32.c (doesnt_want_scroll): Look at both
+       GDK_SCROLL_MASK and GDK_BUTTON_PRESS_MASK. (#139628)
+
 2006-05-09  Matthias Clasen  <mclasen@redhat.com>
 
        * gtk/gtktextbuffer.c (gtk_text_buffer_class_init): Add a :cursor-position

diff --git a/gdk/win32/gdkevents-win32.c b/gdk/win32/gdkevents-win32.c
index 3683bc3a575a1ec65074ed2747ca27a46b9da82e..a6db62f95204a2b9a61021037c71c09bbb52c02f 100644 (file)
--- a/gdk/win32/gdkevents-win32.c
+++ b/gdk/win32/gdkevents-win32.c
@@ -1801,11 +1801,12 @@ static gboolean
 doesnt_want_scroll (gint mask,
                    MSG *msg)
 {
-#if 0
-  return !(mask & GDK_SCROLL_MASK);
-#else
-  return !(mask & GDK_BUTTON_PRESS_MASK);
-#endif
+  /* As there are no separate scroll events in X11, button press
+   * events are used, so higher level code might be selecting for
+   * either GDK_BUTTON_PRESS_MASK or GDK_SCROLL_MASK when it wants GDK
+   * scroll events. Make sure this works in the Win32 backend, too.
+   */
+  return !(mask & (GDK_SCROLL_MASK|GDK_BUTTON_PRESS_MASK));
 }
 
 static void